From: Georges Basile Stavracas Neto <georges.stavracas@gmail.com>
Date: Tue, 11 Feb 2020 13:33:02 +0000 (-0300)
Subject: authPrompt: Only spin on password entries
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~777^2^2~65
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=0f34cbb6588e1d9e7a9b0a077ef20a6ea3f78f18;p=gnome-shell.git

authPrompt: Only spin on password entries

Spinning while typing the password is a bit off-putting, and
inconsistent with how regular authentication behaves.

Only spin the spinner after typing the password.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/922
---

diff --git a/js/gdm/authPrompt.js b/js/gdm/authPrompt.js
index ff04af639..5b6966eb5 100644
--- a/js/gdm/authPrompt.js
+++ b/js/gdm/authPrompt.js
@@ -157,8 +157,9 @@ var AuthPrompt = GObject.registerClass({
             });
 
             entry.clutter_text.connect('activate', () => {
+                let shouldSpin = entry === this._passwordEntry;
                 if (entry.reactive)
-                    this._activateNext();
+                    this._activateNext(shouldSpin);
             });
         });
 
@@ -177,9 +178,12 @@ var AuthPrompt = GObject.registerClass({
         this._defaultButtonWell.add_child(this._spinner);
     }
 
-    _activateNext() {
+    _activateNext(shouldSpin) {
         this.updateSensitivity(false);
-        this.startSpinning();
+
+        if (shouldSpin)
+            this.startSpinning();
+
         if (this._queryingService)
             this._userVerifier.answerQuery(this._queryingService, this._entry.text);
         else
diff --git a/js/gdm/loginDialog.js b/js/gdm/loginDialog.js
index 2505d1894..cad09b548 100644
--- a/js/gdm/loginDialog.js
+++ b/js/gdm/loginDialog.js
@@ -928,7 +928,6 @@ var LoginDialog = GObject.registerClass({
                 let answer = this._authPrompt.getAnswer();
                 this._user = this._userManager.get_user(answer);
                 this._authPrompt.clear();
-                this._authPrompt.startSpinning();
                 this._authPrompt.begin({ userName: answer });
                 this._updateCancelButton();
             });